Skip to content

NamedReroute Nodes#335

Closed
C209 wants to merge 1 commit intoMothCocoon:5.xfrom
C209:c209_pr
Closed

NamedReroute Nodes#335
C209 wants to merge 1 commit intoMothCocoon:5.xfrom
C209:c209_pr

Conversation

@C209
Copy link

@C209 C209 commented Feb 6, 2026

Add NamedReroute Nodes for better graph management.

@MothDoctor
Copy link
Contributor

I'm gonna be honest with you. I don't think I will decide to integrate this into the mainline branch. The fundamental design of the Flow Graph is to keep graphs super-clear. Easy to read, debug, and maintain. This was a conscious decision to enforce having visible links between all nodes. Without that, approaches like "debug by wire" can't be fully utilized.

I'm open to further discussion here and on the plugin's Discord.
I don't close PR, as we can safely assume that quite a few teams would actually ignore the design approach described above. And they would be happy to integrate Named Reroute to their projects :)

@MothDoctor MothDoctor force-pushed the 5.x branch 7 times, most recently from a7ac887 to 5e60d3a Compare February 22, 2026 15:48
@MothDoctor
Copy link
Contributor

Hey @C209, I'm closing this PR. I've talked with community and the consensus confirms my original design: Named Reroutes would break the readability and simplicity of Flow Graph. And the same this would increase the maintenance cost of the system.

But it's clear some teams would like to have Named Reroutes. They would like to know that such a modification exists. So I added a documentation page: Notable Pull Request, and linked to this sumbit there :)
https://mothcocoon.github.io/FlowGraph/Forks/NotablePullRequests

@MothDoctor MothDoctor closed this Mar 7, 2026
@C209
Copy link
Author

C209 commented Mar 9, 2026

I totally understand your design. It's good to leave the implementation here for the teams that need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants